Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Enable Windows device tests on net7.0 #16756

Merged
merged 10 commits into from
Aug 22, 2023
Merged

Enable Windows device tests on net7.0 #16756

merged 10 commits into from
Aug 22, 2023

Conversation

jfversluis
Copy link
Member

@jfversluis jfversluis commented Aug 15, 2023

Description of Change

This brings #15629 and #15885 to net7.0 as well and enabled running the device tests on Windows for the .NET 7 branch. This has already been enabled for the main branch (currently .NET 8)

@samhouts samhouts added this to the .NET 7 + Servicing milestone Aug 15, 2023
@jfversluis jfversluis changed the title Try and enable Windows device tests on net7.0 Enable Windows device tests on net7.0 Aug 16, 2023
@jfversluis jfversluis marked this pull request as ready for review August 16, 2023 13:24
@jfversluis jfversluis requested a review from a team as a code owner August 16, 2023 13:24
@jfversluis jfversluis marked this pull request as draft August 16, 2023 15:11
@jfversluis

This comment was marked as outdated.

@jfversluis

This comment was marked as off-topic.

@jfversluis jfversluis marked this pull request as ready for review August 18, 2023 18:22
@jfversluis
Copy link
Member Author

Apart from the failing test that is unrelated and a more general problem, this one is good now!

@jfversluis

This comment was marked as off-topic.

jfversluis and others added 10 commits August 21, 2023 07:08
* Build & Run Packaged Windows DeviceTests

* Disable running for Core

* Skip failing tests

* Skip Launcher tests

* Update Launcher_Tests.cs

* Update windows.cake

* Revert "Update windows.cake"

This reverts commit f94dfdd.

* Update windows.cake
* Fix WinUI test issues

* - enable core tests

* - core test fixes

* - ignore semantics tests

* - fix

* - syntax errors
@rmarinho rmarinho enabled auto-merge (squash) August 21, 2023 13:19
@jfversluis jfversluis marked this pull request as draft August 21, 2023 14:05
auto-merge was automatically disabled August 21, 2023 14:05

Pull request was converted to draft

@rmarinho
Copy link
Member

@jfversluis seems this is failing on net7 also on iOS , so I think this is good to merge.

@rmarinho rmarinho marked this pull request as ready for review August 21, 2023 16:05
@jfversluis jfversluis enabled auto-merge (squash) August 22, 2023 18:01
@jfversluis jfversluis merged commit da6568b into net7.0 Aug 22, 2023
31 of 33 checks passed
@jfversluis jfversluis deleted the devicetests-net7 branch August 22, 2023 18:06
jfversluis added a commit that referenced this pull request Aug 24, 2023
* Update TestBase.cs

* Revert "Update TestBase.cs"

This reverts commit d761d34.

* Update AssertionExtensions.iOS.cs

* Revert "Update AssertionExtensions.iOS.cs"

This reverts commit f88d9f1.

* Revert "Enable Windows device tests on net7.0 (#16756)"
@github-actions github-actions bot locked and limited conversation to collaborators Dec 7, 2023
@samhouts samhouts added the fixed-in-7.0.96 Look for this fix in 7.0.96 SR8! label Aug 2, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-testing Unit tests, device tests fixed-in-7.0.96 Look for this fix in 7.0.96 SR8! platform/windows 🪟 t/housekeeping ♻︎
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants